Fix stats by country in nginx status page #1235

aledbf · 2016-06-17T22:25:37Z

aledbf · 2016-06-17T22:28:40Z

bprashanth · 2016-06-20T16:41:23Z

ingress/controllers/nginx/nginx.tmpl

@@ -16,18 +16,24 @@ events {
 }

 http {
+    {{/* we use the value of the header X-Forwarded-For to be able to use the geo_ip module */}}
+    real_ip_header      X-Forwarded-For;


for my own education, can it geo locate based on src ip as well?

yes that's the default if no X-Forwarded-For header is present in the request

bprashanth · 2016-06-20T16:41:30Z

LGTM

googlebot added the cla: yes label Jun 17, 2016

Fix stats by country in nginx status page

a152cc4

bprashanth reviewed Jun 20, 2016
View reviewed changes

bprashanth merged commit eeecaa2 into kubernetes-retired:master Jun 20, 2016

aledbf deleted the fix-geoip branch June 20, 2016 21:01

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix stats by country in nginx status page #1235

Fix stats by country in nginx status page #1235

aledbf commented Jun 17, 2016 •

edited

Loading

aledbf commented Jun 17, 2016

bprashanth Jun 20, 2016

aledbf Jun 20, 2016 •

edited

Loading

bprashanth commented Jun 20, 2016

Fix stats by country in nginx status page #1235

Fix stats by country in nginx status page #1235

Conversation

aledbf commented Jun 17, 2016 • edited Loading

aledbf commented Jun 17, 2016

bprashanth Jun 20, 2016

Choose a reason for hiding this comment

aledbf Jun 20, 2016 • edited Loading

Choose a reason for hiding this comment

bprashanth commented Jun 20, 2016

aledbf commented Jun 17, 2016 •

edited

Loading

aledbf Jun 20, 2016 •

edited

Loading